Skip to content

Address a or for the python cli to address the problem where choosing any gro...#2

Merged
7r15 merged 1 commit into
mainfrom
wright/address-a-or-for-the-python-cli-to-address-the-p-ab8cfe71
Jun 16, 2026
Merged

Address a or for the python cli to address the problem where choosing any gro...#2
7r15 merged 1 commit into
mainfrom
wright/address-a-or-for-the-python-cli-to-address-the-p-ab8cfe71

Conversation

@wright-agent

@wright-agent wright-agent Bot commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Draft PR prepared by Wright from a Qortium chat request.

Reporter: QuickMythril

Original request:
@​Wright - please open a PR or Issue for the python cli to address the problem where choosing any group (including 0 or 2) in the chat menu, results in the development group (group 1) being used.

Changed files:

  • qortium_cli/services.py
  • qortium_cli/tools.py
  • qortium_cli/tools/__init__.py
  • tests/test_chat_commands.py
  • tests/test_chat_service.py

Worker summary:
Implemented the chat group selection fix.

Changed:

  • qortium_cli/services.py: build_chat() now accepts tx_group_id and sends it as a /chat request parameter while preserving the existing JSON payload.
  • qortium_cli/tools/__init__.py and qortium_cli/tools.py: chat sends now pass the selected ctx.chat.tx_group_id through explicitly.
  • tests/test_chat_commands.py: regression assertion that selected group 2 reaches the chat builder.
  • tests/test_chat_service.py: new service-level test verifying /chat receives params={"txGroupId": 2}.

Validation:

  • Passed: python3 -m py_compile qortium_cli/services.py qortium_cli/tools/__init__.py qortium_cli/tools.py tests/test_chat_service.py tests/test_chat_commands.py
  • Passed: python3 -m unittest tests.test_chat_service
  • Not run fully: pytest is not installed, and the existing chat command unittest import is blocked by missing rich.

Validation:

  • python -m pytest failed (1): [Errno 2] No such file or directory: 'python'

Notes:

  • This is a draft PR for maintainer review before merge.
  • Wright omitted private runtime details and local paths from this PR body.

@7r15
7r15 marked this pull request as ready for review June 16, 2026 23:21
@7r15
7r15 merged commit 2893ba5 into main Jun 16, 2026
2 checks passed
@7r15
7r15 deleted the wright/address-a-or-for-the-python-cli-to-address-the-p-ab8cfe71 branch June 17, 2026 17:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants